Skip to content
This repository has been archived by the owner on May 17, 2024. It is now read-only.

Update README.md, and FundMe.test.js #61

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Update README.md, and FundMe.test.js #61

wants to merge 2 commits into from

Conversation

stackaccount1
Copy link

add hardhat-deploy install command for easier grab and install while completing tutorial

add hardhat-deploy install command for easier grab and install while completing tutorial
Change line 126 due to error:  
 AssertionError: Expected transaction to be reverted, but other exception was thrown: RangeError: Maximum call stack size exceeded

// Make a getter for storage variables
```
await expect(fundMe.getFunder(0)).to.be.reverted
```
Changed to 

```
await expect(fundMe.getFunder({ value: 0 })).to.be.reverted
```
This worked for me if this is correct syntax, it probably should be changed
@stackaccount1 stackaccount1 changed the title Update README.md Update README.md, and FundMe.test.js Aug 19, 2022
@stackaccount1 stackaccount1 closed this by deleting the head repository Aug 19, 2022
@stackaccount1 stackaccount1 reopened this Aug 22, 2022
@PatrickAlphaC
Copy link
Owner

Thanks!

But why this line:

 await expect(fundMe.getFunder({ value: 0 })).to.be.reverted

@Nicolaswg
Copy link

In my case the fund functions besides that have the required it pass the test with a value of 0 kind of strange this error

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants